Skip to content

Conversation

@thg2k
Copy link
Contributor

@thg2k thg2k commented Jan 29, 2026

See also #20933

Copy link
Member

@TimWolla TimWolla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm okay with this for the existing branches if the changes are simple, but it should also fix PHP-8.4 then.

@thg2k
Copy link
Contributor Author

thg2k commented Jan 29, 2026

I'm okay with this for the existing branches if the changes are simple, but it should also fix PHP-8.4 then.

I sure will, but there the only problem is str_ends_with, which is no longer used in PHP-8.5, so I thought I thought I would start from this branch.

I'll also send the PR for 8.4, then you can figure in which order to apply them.

@TimWolla
Copy link
Member

so I thought I thought I would start from this branch.

Bugfixes are always applied to the lowest applicable branch. I understand that in this case it's two separate issues with both of them affecting only one branch, but nevertheless fixing the older one first seems to be the correct choice.

@thg2k thg2k changed the title gen_stub: Fix compatibility with php 7.4 gen_stub: Fix compatibility with php 7.4 (in PHP-8.5+) Jan 29, 2026
@thg2k
Copy link
Contributor Author

thg2k commented Jan 29, 2026

so I thought I thought I would start from this branch.

Bugfixes are always applied to the lowest applicable branch. I understand that in this case it's two separate issues with both of them affecting only one branch, but nevertheless fixing the older one first seems to be the correct choice.

Sounds right, I opened another PR for 8.4, I guess you could also do some dark magic during merge and apply that one in 8.4 and in phase of conflict resolution reset and apply this one. The only problem with burying changes in a merge commit is that they don't show up in git log -p, which is unfortunate.

@TimWolla TimWolla self-assigned this Jan 29, 2026
@TimWolla TimWolla merged commit 7ec14e3 into php:PHP-8.5 Jan 29, 2026
17 of 18 checks passed
TimWolla added a commit that referenced this pull request Jan 29, 2026
* PHP-8.5:
  gen_stub: Fix compatibility with php 7.4 (in PHP-8.5+) (#21075)
@thg2k thg2k deleted the pr/genstub branch January 29, 2026 11:11
@thg2k
Copy link
Contributor Author

thg2k commented Jan 29, 2026

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants